Skip to content

chore: standard replacement from dotenv to dotenvx #8216

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
sriramveeraghanta merged 1 commit into from
Dec 2, 2025
Merged

chore: standard replacement from dotenv to dotenvx #8216

sriramveeraghanta merged 1 commit into from
Dec 2, 2025

Conversation

@sriramveeraghanta
Copy link
Member

@sriramveeraghanta sriramveeraghanta commented Dec 2, 2025 •
edited by coderabbitai bot
Loading

Summary by CodeRabbit

  • Chores
    • Updated environment configuration dependencies across all applications to improve system consistency and performance while maintaining existing functionality and behavior.

✏️ Tip: You can customize this high-level summary in your review settings.

Copilot AI review requested due to automatic review settings December 2, 2025 09:59
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 2, 2025 •
edited
Loading

Caution

Review failed

The pull request is closed.

Walkthrough

This pull request migrates from the dotenv package to @dotenvx/dotenvx across multiple applications (admin, live, space, web). Updates include replacing dependencies in package manifests, modifying vite configuration imports, and adding the new package to the pnpm workspace catalog. Runtime behavior is preserved.

Changes

Cohort / File(s) Summary
Package dependency updates
apps/admin/package.json, apps/web/package.json, apps/space/package.json		 Replaced devDependency dotenv (^16.4.5) with @dotenvx/dotenvx (catalog:)
Package dependency updates
apps/live/package.json		 Replaced dependency dotenv with @dotenvx/dotenvx (catalog:)
Vite configuration imports
apps/admin/vite.config.ts, apps/space/vite.config.ts, apps/web/vite.config.ts		 Updated dotenv imports to use @dotenvx/dotenvx; preserved dotenv.config() call
Workspace catalog
pnpm-workspace.yaml		 Added @dotenvx/dotenvx version 1.51.1 to catalog section

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • Repetitive, homogeneous changes following a consistent migration pattern across multiple files
  • No logic modifications or behavioral changes—purely dependency and import swaps
  • Configuration-focused updates with minimal risk

Poem

🐰 Hop along with dotenvx so new,
From dotenv old, we bid adieu!
Config files dance in harmony,
Workspace catalog set so free!
🌱✨

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore-deps-dotenv
📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e07d91e and c9271df.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (8)
  • apps/admin/package.json (1 hunks)
  • apps/admin/vite.config.ts (1 hunks)
  • apps/live/package.json (1 hunks)
  • apps/space/package.json (1 hunks)
  • apps/space/vite.config.ts (1 hunks)
  • apps/web/package.json (1 hunks)
  • apps/web/vite.config.ts (1 hunks)
  • pnpm-workspace.yaml (1 hunks)

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@sriramveeraghanta sriramveeraghanta merged commit ec478a8 into preview Dec 2, 2025
9 of 10 checks passed
@sriramveeraghanta sriramveeraghanta deleted the chore-deps-dotenv branch December 2, 2025 10:02
Copilot AI reviewed Dec 2, 2025
View reviewed changes
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR migrates the project from the standard dotenv package to @dotenvx/dotenvx for enhanced environment variable management across the monorepo. The migration standardizes the dependency using pnpm's catalog feature to ensure version consistency (1.51.1) across all apps (web, space, admin, and live). The API remains compatible as dotenvx internally uses dotenv and maintains the same .config() method signature.

Key Changes

  • Added @dotenvx/dotenvx version 1.51.1 to the pnpm catalog for centralized version management
  • Updated import statements from default imports to namespace imports across all vite config files
  • Migrated apps/live from a standalone version (1.49.0) to the catalog version (1.51.1)

Reviewed changes

Copilot reviewed 8 out of 9 changed files in this pull request and generated no comments.

Show a summary per file
File Description
pnpm-workspace.yaml Added @dotenvx/[email protected] to the catalog for centralized version management
pnpm-lock.yaml Updated lockfile with new @dotenvx/dotenvx version and removed direct dotenv dependencies
apps/web/vite.config.ts Changed import from default to namespace import for @dotenvx/dotenvx
apps/web/package.json Replaced dotenv with @dotenvx/dotenvx as a dev dependency using catalog reference
apps/space/vite.config.ts Changed import from default to namespace import for @dotenvx/dotenvx
apps/space/package.json Replaced dotenv with @dotenvx/dotenvx as a dev dependency using catalog reference
apps/admin/vite.config.ts Changed import from default to namespace import for @dotenvx/dotenvx
apps/admin/package.json Replaced dotenv with @dotenvx/dotenvx as a dev dependency using catalog reference
apps/live/package.json Updated from standalone version 1.49.0 to catalog reference for version 1.51.1
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

ClarenceChen0627 pushed a commit to ClarenceChen0627/plane that referenced this pull request Dec 5, 2025
@sriramveeraghanta @ClarenceChen0627
chore: standard replacement from dotenv to dotenvx (makeplane#8216)
6f6c433
ClarenceChen0627 pushed a commit to ClarenceChen0627/plane that referenced this pull request Dec 5, 2025
@sriramveeraghanta @ClarenceChen0627
chore: standard replacement from dotenv to dotenvx (makeplane#8216)
6a3ea1f
ClarenceChen0627 pushed a commit to ClarenceChen0627/plane that referenced this pull request Dec 5, 2025
@sriramveeraghanta @ClarenceChen0627
chore: standard replacement from dotenv to dotenvx (makeplane#8216)
36d2c48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@pushya22 pushya22 pushya22 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@sriramveeraghanta @pushya22